Add ainModule #39
Add ainModule #39
Conversation
|
login logout상태 확인을 위해 isLogIn 같은 필드가 하나 있으면 좋을 것 같습니다. |
|
LGTM with comments |
login 관련은 ainize 에서 진행할 것 같아 setDefaultAccount만 우선 넣어놨는데 ain에서 진행하는게 좋을까요? 우선 getDefaultAccount 함수는 추가하겠습니다. @akastercomcom |
isLoggIn을 이용해야 model에서 login했을 경우 사용하는 함수들 체크를 할 수 있을 것 같습니다. |
src/ain.ts
Outdated
| } | ||
|
|
||
| setDefaultAddress(privateKey: string) { | ||
| if(!this.checkAinInitiated()) |
There was a problem hiding this comment.
error를 던지는 부분까지 AinInitated에 포함시키는 건 어떨까요?
|
isLogin 하나만 추가해 주시고 머지 부탁드립니다!( model 구현할 때 필요합니다) |
|
isDefaultAccountExist 추가하여 login 여부 판단할 수 있도록 변경하였습니다. ainize / model 에서 이 함수 호출하여 로그인 여부 관리하면 좋을 것 같습니다. |
|
Thanks for the review! |
No description provided.